Hi @petk, here's another build-related PR. Technically this PR addresses two separate concerns:

• libphp.* doesn't respect --libdir. It always installs to $prefix/lib. To me this seems like a small bug.
• php-config doesn't have enough info to confidently locate libphp. If a user knows that the build always installs to $prefix/lib, they can of course use -L$(php-config --prefix)/lib, but for those who don't know that bit of trivia, it feels like a guess.

Bump @petk

I'm onto this in the following days, thanks for your patience...

I'll just recheck this a bit further but overall it's all ok. Yes, I totally agree that php-config script needs to have also embed info. Good suggestions.

I'll just add note here to not forget when merging this... The approximate change for the php-config man page:

diff a/scripts/man1/php-config.1.in b/scripts/man1/php-config.1.in
--- a/scripts/man1/php-config.1.in
+++ b/scripts/man1/php-config.1.in
@@ -39,6 +39,14 @@ Directory where extensions are searched by default
 Directory prefix where header files are installed by default
 .TP
 .PD 0
+.B \-\-lib-dir
+Directory where libraries are installed by default
+.TP
+.PD 0
+.B \-\-lib-embed
+PHP embed library name
+.TP
+.PD 0
 .B \-\-php-binary
 Full path to php CLI or CGI binary
 .TP

Thank you @petk. I've incorporated your changes.

@adsr thanks for your patience. Yes, it is taking abnormally long :) Issue is that I'm still checking how everything fits together. And merge is coming up soon here, no worries.

What I'm currently also thinking is adding additional pkgconf template for PHP. Like an optional add-on:

build/php.pc.in:

prefix=${prefix}
exec_prefix=${exec_prefix}
includedir=${prefix}/include...
libdir=${exec_prefix}/lib

Name: php
Description: PHP, a general-purpose scripting language
Version: ${php_version}
Cflags: -I${includedir}/php/...
Libs: -L${libdir} -lphp

So, it can be used together with pkgconf command-line utility. If PHP extensively already uses pkgconf for many dependencies, it should also provide pkgconf template for installation, I believe.

And, I'll probably just sync these libdir names a bit. But this is just for the info, nothing to change in the PR...

Thanks @adsr added to master branch (upcoming into PHP-8.4). We'll sync this further with pkgconf and the rest of the options in the future so this embed SAPI can be perhaps a bit easier to use.